refactor trackfilter.

use std::sort, stable_sort instead of qsort.
use QList instead of malloc'd arrays.
minimize the use of duplicated information (track_ct, track_pts, timeless_pts
first_time, last_time), and maintain the remaining
duplicated information (track_list, a possible sorted version of my_track_head).
use auto some more.
eliminate some redundant casting.
eliminate atoi.

in trackfilter remove redundant casts and use auto.

trackfilter bools and case insensitive opts.

refactor track filter.

user visible changes:
Previously it was possible to cause a segmentation fault (due to a double delete)
if all the tracks were filtered out by the stop and start options.
This has been corrected.

Previously UTC time was used with the title option for split operations,
but local time was used with the title option for the pack operations.
Now UTC time is always used with the title option.

Previously the split and sdistance options allowed an option value
consisting of only the units.  This was undocumented.  Now both a number
and the units are required.

Previously the split option documentation stated that units of seconds
would be used if none were specified, however the units were actually required.
The documenation is updated to reflect historical and current usage.

Previously the split and sdistance options allowed the units to precede or
follow the quantity.  The documentation stated the units must follow the
quantity.  Now the units must follow the quantity.

Previously synthesized speeds ignored the fractional part of the track point times,
now the fractional part of the track point times are used in the speed calculation.

refactoring changes:
time_t is replaced where possible by gpsbabel::DateTime or QDateTime.

option value parsing is done with QRegularExpression.

some integer types were converted to bool types.

test cases are expanded for better coverage.

Corrected handling elevation and speed (#291)

* Corrected handling elevation and speed

* Added reference files for testing converting fit-file from Garmin Oregon 700 to gpx-file

* Added test of converting fit-file from Garmin Oregon 700 to gpx-file

add include guard for src/core/optional.h (#290)

clazy cleanup for gui (#289)

Merge pull request #288 from tsteven4/lat_dir_input

enhance xcsv reader to not ignore LAT_DIR,LON_DIR.

Merge pull request #287 from tsteven4/lat_dir

fix up style file OFIELD option processing.

enhance xcsv reader to not ignore LAT_DIR,LON_DIR.

this came up on [Gpsbabel-misc] Globalsat DG-500.

fix up style file OFIELD option processing.

Merge pull request #285 from gpsbabel/atoi

Atoi: rely less on atoi, scanf, other cleanups. Safer parsing in stmsdf in particular.

Merge branch 'master' into atoi

Merge pull request #284 from gpsbabel/cl-cleanups

CodeLion-inspired and manual cleanups

add support for multiple urls for route headers. (#276)

* add support for multiple urls for route headers.

add support in gpx for gpx/rte/link, gpx/rte/url, gpx/rte/urlname.
add support in gpx for gpx/trk/link, gpx/trk/url, gpx/trk/urlname.

Note that the gpx writer can violate the schema when fprint_xml_chain
is used to echo unrecognized elements from input to output.
This is because the gpx 1.0/1.1 schema requires an xsd:sequence of
elements, i.e. the order of the elements is specified, and any
elements output by fprint_xml_chain won't necessarily be in
the correct order with other elements that are specifically
handled in the gpx writer.

By processing additional gpx elements as listed above on read we
prevent them from causing schema violations on write.  However,
other unrecognized elements can still cause schema violations.

* make common link code common in gpx reader.

Don't #include debug

A near rewrite of the reader of the stmsdf reader to more effectively parse fields as QDate,Times,Floats, etc. Still not very readable, but should be more bulletproof for bad input.

Don't name params in decls in itracku

Sign extension fix in EasyGPS

Remove xstrrstr and BASE_STRUCT from defs.h

Remove write-only variable in compegps.

Drop some unused method protos from CET.

XmlStreamWriter: make single arg methods explicit. Drop unused methods.

Removing scraggling bits of xhtmlent from build files.

Remove (now) unused xhtmlent table.

Make types in ggv_bin more safe for 32/64 systems.

Default construct to eliminate warning in geojson

Build fix for Cmake/CLion on Mac. Correct handling of frameworks. They're like
libs, but not...

add cppcheck target to GPSBabel.pro. (#280)

* add cppcheck target to GPSBabel.pro.

fix a few cppcheck errors.

* fix a few cppcheck items.

strptime fix is from upstream.

Wrap (use) of variables in #defines.

Warning fix in raymarine

Warning fix in locus

Fix warning in maggeo

Warning fixes in jeeps: move use closer to initialization so that the
test of device is effective, don't set locals before return.

Tame string formatting warning in libusb.

Remove unneeded const qualifier on pass-by-value types.

Replace size() == 0 with empty() when more readable.

Const-ipate jeeps function signature. (Yeah, coz THAT makes that API beautiful...)

Automated cleanups for passing by reference and fixing pointer ownership on calls.

Modernize loop iterators to walk arrays and containes instead of independently
maintained constants.

Modernize C style explicit (void) in decls to (), eliminate return
at end of bunction bodies.

Modernize bool use of true/false.

Make prototype arg names match function arg names.

Pick up a few old NULL->nullptr conversions that are stuck inside __APPLE that
tsteven4 dodged around since he couldn't confirm.

For single argument ctors, mark type explicit.

If there's only one argument in a ctor, mark it explicit to make it less
easy to fall into implicit conversions.

Diff and Terrain in geocaches are integers internally.

generate html coverage reports. (#281)

* generate html coverage reports.

* tweak coverage recipe.

remove obsolete msvc workarounds. (#278)

msvc 2012 and earlier are not supported.
They didn't support c++14 anyway which we currently require.

use QThread::usleep instead of our own function. (#277)

Our test suite doesn't test sleep, but the nmea pause option
can be used on write to verify functionality interactively.

obsolete DEBUG_MEM (#279)

simplify conversion from utm. (#275)

* simplify conversion from utm.

add test cases for Norway and Svalbard exceptions.

* Correct bug in computing latitude band X.

enhance test to cover exceptional width of band X.

update shapelib to 1.4.1 (#274)

enhance sort filter for rtes & trks. (#273)

use bool, true, false. (#272)

add msvc2017 builds to appveyor. (#270)

update docker build script to use latest bionic. (#269)

Merge pull request #266 from tsteven4/clazy1

Incorporate improvements found by Qt oriented code checker clazy

Merge pull request #267 from gpsbabel/gpsbabel-patch-2

Remove leftover code from 'adding class' change

Remove leftover code from 'adding class' change

As tsteven4 points out, No reasing to move is_internal to local and back

fix clazy detected 'use isEmpty() instead'

this is from -Wclazy-isempty-vs-count

fix clazy detected 'Use *Ref() instead'

These are from -Wclazy-qstring-ref, and were automatically fixed.

fix clazy detected 'unused *'

these are from -Wclazy-unused-non-trivial-variable

fix clazy detected 'allocating an unneeded temporary container'

these are from -Wclazy-container-anti-pattern

Also, consistently use range based for loops in csv_util.cc

fix clazy detected 'c++11 range-loop might detach Qt container'

these are from -Wclazy-range-loop

fix clazy detected 'Use multi-arg instead'

these are from -Wclazy-qstring-arg

update travis for xenial (#265)

* update travis to use xenial.

add new travis target to build on xenial.
switch travis coverage generation to xenial.
update codacy upload tool for compatibility with newer java on xenial.

* fix gcov issues with "cannot open notes file"

support kml 2.3 track element. (#264)

* support kml 2.3 track element.

relax ordering requirement for gx:Track when and
coord children.

* fix kml includes and indentation.

Merge pull request #261 from gpsbabel/gpsbabel-patch-2

Update dependencies, notably for <optional>

Replace queues with Qlist in session handling. (#263)

session tracks a possible series of input formats and input filenames.

Merge pull request #262 from tsteven4/tcache

extend travis mac cache timeout.

extend travis mac cache timeout.

Merge pull request #260 from tsteven4/cpp14_msvc

move to c++14 standard.

Merge pull request #259 from tsteven4/queue_bye_bye

eliminate queues in csv style handling.

Update dependencies, notably for <optional>

Merge pull request #256 from tsteven4/rpath

have configure conditionally set rpath.

move to c++14 standard.

eliminate queues in csv style handling.

also:
fix bug with style option encoding.
pass some parameters by const reference.
eliminate some unused time fiddling.

Merge pull request #258 from tsteven4/sortbug

correct sort filter time option to use msecs.

correct sort filter time option to use msecs.

use consistent whitespace in configure comments.

have configure conditionally set rpath.

this may be necessary when using a Qt installation
that was not provided by the system.

Merge pull request #255 from gpsbabel/parallel

Parallelize CLI and GUI make.

Parallelize CLI and GUI make. testo and vtesto are harder.

Merge pull request #254 from gpsbabel/tweaks

Move more xcsv processing into QStrings. Fewer conversions

Allow string to be modified during parse. Ugh.

Let xcsv parser keep QStrings up for one more level.

Push QStrings -> char * conversions down one more level.

Merge pull request #253 from gpsbabel/tweaks

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: G…

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: GNU sed is /usr/local/bin/

Merge pull request #252 from gpsbabel/tweaks

Find gsed when doing a tools/docfix on a Mac populated by Homebrew: GNU sed is /usr/local/bin/

Merge pull request #249 from codacy-badger/codacy-badge

Add a Codacy badge to README.md

Add codacy badges to Github README.md

Add Codacy badge

Merge pull request #246 from tsteven4/trackdata

introduce optional class

introduce optional class

for things that may or may not have a value.
demonstrate usage with computed_trkdata.

Merge pull request #245 from tsteven4/exif2

a few exif cleanups.

a few exif cleanups.

Merge pull request #230 from tsteven4/bng

allow unicsv bng input to use all numeric grid references.

tweak unicsv bng documentation thanks Hornbydd.